opt: add PlanGramMismatchPenalty cost penalty flag#169717
opt: add PlanGramMismatchPenalty cost penalty flag#169717trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
Conversation
|
😎 Merged directly without going through the merge queue, as the queue was empty and the PR was up to date with the target branch - details. |
|
This change is |
|
First commit is #169197 and can be ignored. |
michae2
force-pushed
the
plangram-mismatch-penalty
branch
2 times, most recently
from
1760f15 to
0fbe295
Compare
|
Detected infrastructure failure (matched: self-hosted runner lost communication with the server). Automatically rerunning failed jobs. (run link) |
michae2
force-pushed
the
plangram-mismatch-penalty
branch
from
0fbe295 to
f1c1ca4
Compare
|
Detected infrastructure failure (matched: self-hosted runner lost communication with the server). Automatically rerunning failed jobs. (run link) |
1 similar comment
|
Detected infrastructure failure (matched: self-hosted runner lost communication with the server). Automatically rerunning failed jobs. (run link) |
DrewKimball
left a comment
DrewKimball
left a comment
There was a problem hiding this comment.
@DrewKimball reviewed 7 files and all commit messages, and made 1 comment.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on ZhouXing19).
Add a new PlanGramMismatchPenalty to the cost penalty bitmask. This penalty will be applied when a plan does not match the required PlanGram pattern. Its precedence is stronger than UnboundedCardinalityPenalty but weaker than FullScanPenalty, so that inline hints (which set FullScanPenalty or HugeCostPenalty) take precedence over PlanGram matching. Informs: cockroachdb#152053 Release note: None Co-Authored-By: roachdev-claude <[email protected]>
michae2
force-pushed
the
plangram-mismatch-penalty
branch
from
f1c1ca4 to
8c540ce
Compare
|
Thanks! /trunk merge |
3 participants
Add a new PlanGramMismatchPenalty to the cost penalty bitmask. This penalty will be applied when a plan does not match the required PlanGram pattern. Its precedence is stronger than UnboundedCardinalityPenalty but weaker than FullScanPenalty, so that inline hints (which set FullScanPenalty or HugeCostPenalty) take precedence over PlanGram matching.
Informs: #152053
Release note: None
Co-Authored-By: roachdev-claude [email protected]